Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Guess who - game #343

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Guess who - game #343

wants to merge 10 commits into from

Conversation

FridaKristian
Copy link

No description provided.

…tion that loads all the characters and setSecret-function that decides which person is the who.
…ded, the value and it's category will be saved in an object called currectQuestion. An eventlistener is added to see when the user changes her selection
…t of their guess. The array of charactersInPlay is then filtered based on the result of the guess. The remainin characters in charactersInPlay is regenerated to the board with the function generateBoard
…hen it is pressed the user is asked to confirm (since game will end after guess. Then the name of the secret person is compared to the guessed name. An alertmessage is shown. To do for tomorrow is to show a win or lose section and hide the board game.
…-lose board is shown by changing display:none to display:flex. Then a win or lose message is shown by changing innerHTML. Then the gameboard is removed by changing display: flex to display:none
…-button is pressed (shown on the win-or-lose board)
…the question section and updates each time 'Find out'-button is pressed
Copy link

@FridafridaL FridafridaL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Frida! Great work with your 'Guess who'-project. The code looks nice and clear. I like that you manage to make the code short, fort example in the selectQuestion-function. But still understandable. The game works as it should in the browser and the fallout can be both winning or losing :)
You've also made a guess-counter which was in the stretch goals. Thumbs up!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants